Skip to content

feat(ops): P1-3 clarify fallback routing before react - #234

Merged
Cyning12 merged 4 commits into
mainfrom
task/ops-chat-session-sink-p0-p1
Jul 9, 2026
Merged

feat(ops): P1-3 clarify fallback routing before react#234
Cyning12 merged 4 commits into
mainfrom
task/ops-chat-session-sink-p0-p1

Conversation

@Cyning12

@Cyning12 Cyning12 commented Jul 9, 2026

Copy link
Copy Markdown
Owner

Summary

  • 新增 api/ops/orchestrator/clarify.pyclarify_if_fallback(query, session_id, transcript, slots),支持 LLM 1 轮澄清、规则兜底、LLM 失败降级 ReAct fallback
  • api/ops/chat_service.py:FALLBACK 分支先 clarify 再路由;needs_clarification=true 返回澄清卡片并记录 clarify.asked event;false 时按补齐后的 intent/slots 路由到 fast/deep/react
  • api/ops/orchestrator/__init__.py:导出 ClarifyResultclarify_if_fallback
  • 复用 P0-3 load_chat_transcript 为 clarify 提供多轮上下文
  • 减少默认 #545:FALLBACK 不再 silent 进入 ReAct fallback

Test plan

  • pytest tests/ops tests/ops_desk -m "not intent_eval and not intent_benchmark" -q && ruff check api/ops
  • 310 passed, 10 skipped, 1 warning
  • ruff 全绿

Scope

仅 P1-3;未改 P1-4 LLM Router、P1-1 artifact、P1-2 checkpoint、Session 生产图、Agently lab、前端。


Harness 30/40/50 全链通过;HG-TASK-DRAFT / HG-AUDIT-R1 approved。

CyningMM added 4 commits July 9, 2026 19:36
- Add api/ops/orchestrator/clarify.py with clarify_if_fallback
- FALLBACK in chat_service now clarifies first; routes to fast/deep/react/clarify
- Add tests/ops/test_clarify.py and tests/ops/test_chat_service.py
- Adapt tests/ops_desk/test_react_fallback.py fixture to bypass clarify for ReAct tests
40 self-check invoke snapshot and 50 next-hat prompt for P1-3 Clarify routing.
将 50 下一棒 Prompt 中的 40 commit 占位符替换为实际 hash。
…ceptance

R1 review落盘 + task自检结论回填;验证命令310 passed/10 skipped + ruff全绿。
@Cyning12
Cyning12 merged commit 13553de into main Jul 9, 2026
6 of 9 checks passed
@vercel

vercel Bot commented Jul 9, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
ai-ink-brain-api-python Building Building Preview, Comment Jul 9, 2026 11:51am

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants